Skip to content

fix: unread indicator label presence in message list#3031

Merged
khushal87 merged 3 commits intodevelopfrom
fix-unread-indicator-label-presence
Mar 20, 2025
Merged

fix: unread indicator label presence in message list#3031
khushal87 merged 3 commits intodevelopfrom
fix-unread-indicator-label-presence

Conversation

@khushal87
Copy link
Copy Markdown
Contributor

@khushal87 khushal87 commented Mar 19, 2025

The PR is responsible for fixing the current unread indicator pill visibility on the Message List.

  • When you send a lot of messages post the viewable area, the indicator appeared which was buggy. This is now fixed.
  • When you send a single long message in the message list that is more than the visible area the indicator appeared. It is fixed as well.
  • When you are somewhere above in the message list, when you scroll to bottom, you would expect to see the unread messages label in the message list on top of the first unread received message. This is fixed as well.

Linear issue - https://linear.app/stream/issue/RN-158/investigate-faulty-appearance-of-unread-indicator-label-in-message

Comment thread package/src/components/Channel/Channel.tsx
@Stream-SDK-Bot
Copy link
Copy Markdown
Contributor

Stream-SDK-Bot commented Mar 19, 2025

SDK Size

title develop branch diff status
js_bundle_size 467 KB 467 KB 0 B 🟢

Comment thread package/src/components/MessageList/MessageList.tsx
Comment thread package/src/components/MessageList/MessageList.tsx
Comment thread package/src/components/MessageList/MessageList.tsx
Comment thread package/src/components/MessageList/MessageList.tsx Outdated
@khushal87 khushal87 merged commit 53eeafb into develop Mar 20, 2025
5 checks passed
@khushal87 khushal87 deleted the fix-unread-indicator-label-presence branch March 20, 2025 11:03
@github-actions github-actions bot mentioned this pull request Mar 27, 2025
6 tasks
@stream-ci-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 6.7.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants